-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix documentation for PhasedXZGate #6870
Conversation
Hello old friend CirqBot |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6870 +/- ##
=======================================
Coverage ? 97.86%
=======================================
Files ? 1084
Lines ? 94299
Branches ? 0
=======================================
Hits ? 92289
Misses ? 2010
Partials ? 0 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable. I know this means the website needs to be updated too, and will make sure that happens.
Avoid spurious one-character subscripts and superscripts. Follow-up to quantumlib#6870
Avoid spurious one-character subscripts and superscripts in HTML docs. Follow-up to #6870
See documentation on website:
https://quantumai.google/reference/python/cirq/PhasedXZGate?hl=en
This makes the it not subscript the "r"s. I kept this as "math font" instead of making the \tt which is what I would normally due, as this seem to be the standard throughout docs and makes the docstrings more readable.